Skip to content

Fix clang-format version check for 11.1.0 - #9732

Closed
davidwendt wants to merge 1 commit into
NVIDIA:branch-22.02from
davidwendt:fix-clang-format-check
Closed

Fix clang-format version check for 11.1.0#9732
davidwendt wants to merge 1 commit into
NVIDIA:branch-22.02from
davidwendt:fix-clang-format-check

Conversation

@davidwendt

@davidwendt davidwendt commented Nov 19, 2021

Copy link
Copy Markdown
Contributor

Fixes CI builds failing style checks which stop the build from compiling and executing tests.

@davidwendt davidwendt added bug Something isn't working 3 - Ready for Review Ready for review by team libcudf blocker libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change labels Nov 19, 2021
@davidwendt davidwendt self-assigned this Nov 19, 2021
@davidwendt
davidwendt requested a review from a team as a code owner November 19, 2021 13:38
@bdice

bdice commented Nov 19, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

I created #9730 to solve the same problem by forward-merging @galipremsagar’s PR #9716. I think the forward merger is the desired way to fix this?

@davidwendt

davidwendt commented Nov 19, 2021

Copy link
Copy Markdown
Contributor Author

I created #9730 to solve the same problem by forward-merging @galipremsagar’s PR #9716. I think the forward merger is the desired way to fix this?

Perhaps but then why is it failing in 22.02? Seems like something is wrong. This change unblocks me on my current PRs and does not appear to hold up 9730.

@bdice

bdice commented Nov 19, 2021

Copy link
Copy Markdown
Contributor

I created #9730 to solve the same problem by forward-merging @galipremsagar’s PR #9716. I think the forward merger is the desired way to fix this?

Perhaps but then why is it failing in 22.02? Seems like something is wrong. This change unblocks me on my current PRs and does not appear to hold up 9730.

@galipremsagar’s PR targeted 21.12 but had a merge conflict with 22.02, causing the error we see on 22.02. The forward-merger I created (with resolved conflicts from #9721) also includes the change in this PR.

bdice
bdice reviewed Nov 19, 2021
View reviewed changes

@bdice bdice left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR does not correct the version numbers in the conda environment pinnings, so local environments created from that file would not match the pinned clang-format in CI. Forward-merging #9730 would fix the issue while keeping environments consistent.

@davidwendt

davidwendt commented Nov 19, 2021

Copy link
Copy Markdown
Contributor Author

Fixed by #9730

@davidwendt davidwendt closed this Nov 19, 2021
@davidwendt
davidwendt deleted the fix-clang-format-check branch November 22, 2021 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bdice bdice bdice left review comments
@ttnghia ttnghia ttnghia approved these changes
+1 more reviewer
@jrhemstad jrhemstad jrhemstad approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@davidwendt davidwendt

Labels

3 - Ready for Review Ready for review by team bug Something isn't working libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@davidwendt @bdice @ttnghia @jrhemstad